fix: docker-compose.prod.yml 경로 및 서버 설정 동기화#56
Merged
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Code Review
This pull request updates the docker-compose.prod.yml configuration by modifying volume paths for the fluent-bit service, removing the public port mapping for the postgres service, and updating the environment file path. A review comment suggests verifying the fluent-bit volume paths, specifically highlighting a potential typo in the directory name and questioning the relative path logic compared to the previous configuration.
Comment on lines
+6
to
+7
| - ../fluentbit/fluent-bit.conf:/fluent-bit/etc/fluent-bit.conf:ro | ||
| - ../fluentbit/parsers.conf:/fluent-bit/etc/parsers.conf:ro |
There was a problem hiding this comment.
fluent-bit 설정 파일의 경로가 ../fluentbit/로 변경되었습니다. 다음 사항들을 확인해 주세요.
- 디렉토리 이름 오타 가능성: 기존에는
fluent-bit(하이픈 포함)이었으나 현재fluentbit(하이픈 없음)으로 변경되었습니다. 실제 디렉토리 이름이 변경된 것이 아니라면 경로 오류가 발생합니다. - 상대 경로 확인:
docker-compose.prod.yml파일이docker/디렉토리 내부에 있으므로, 설정 파일이docker/fluent-bit/에 위치한다면../fluentbit/가 아닌./fluent-bit/가 올바른 경로입니다.
만약 설정 파일들을 프로젝트 루트의 fluentbit/ 디렉토리로 이동시킨 것이라면 현재 설정이 맞지만, 그렇지 않다면 수정이 필요합니다.
- ./fluent-bit/fluent-bit.conf:/fluent-bit/etc/fluent-bit.conf:ro
- ./fluent-bit/parsers.conf:/fluent-bit/etc/parsers.conf:ro
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 작업 내용 (Description)
🔄 변경 유형 (Type of Change)
✅ 체크리스트 (Checklist)